Skip to content

Conversation

redstrate
Copy link
Contributor

This is supposed to be emitted whenever the user performs an intentional interaction, which works whenever you scroll the Slint Flickable with the scrollwheel. Dragging the scrollbar on the Qt style didn't do anything, since that was never hooked up to this callback.

I only implemented this in the Qt style, because that's the only one that uses the NativeScrollView element.

See #9574.

This is supposed to be emitted whenever the user performs an
intentional interaction, which works whenever you scroll the Slint
Flickable with the scrollwheel. Dragging the scrollbar on the Qt
style didn't do anything, since that was never hooked up to this
callback.

I only implemented this in the Qt style, because that's the only
one that uses the NativeScrollView element.

See slint-ui#9574.
Copy link
Member

@ogoffart ogoffart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. This is indeed a bugfix and that seems correct.

This kind of behavior could benefit from a test in the tests/cases/widgets folder.
(For example this test checks what happens when scrolling on a SpinBox

)

@ogoffart ogoffart merged commit 7d038eb into slint-ui:master Oct 7, 2025
41 checks passed
@redstrate
Copy link
Contributor Author

Ah you merged it already, I didn't mention that I'm working on a test! 😄 I'll just add it in another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants